Codecov Report

Merging #90 (ff625fa) into main (4d34cd2) will decrease coverage by 0.13%.
Report is 6 commits behind head on main.
The diff coverage is 82.00%.

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

@@            Coverage Diff             @@
##             main      #90      +/-   ##
==========================================
- Coverage   83.18%   83.05%   -0.13%     
==========================================
  Files          11       11              
  Lines        1540     1688     +148     
==========================================
+ Hits         1281     1402     +121     
- Misses        259      286      +27     

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

Preview page for your plugin is ready here:
https://preview.napari-hub.org/DeepLabCut/napari-deeplabcut/90
Updated: 2023-08-31T08:05:01.184269

Probably a couple of tweaks that could be made here and there; but that works very smoothly even for maDLC.

Another thing that should be tweaked @jeylau :

They y_lim should be independent of the dataset and dependent on the plot window (or adjustable)
Indeed, keypoint jumps can squeeze the window depending on the dataset's quality

Good point; one thing that could be added then is a NavigationToolbar2QT, which would allow panning, zooming, etc.

Is this superseded by #92 ?

@MMathisLab, these 2 PRs are distinct components. I will add the toolbar Seb mentioned above so this one is ready 😊

Also, regarding the warnings in the console, they will be removed with napari's next release and vispy >= 0.13 napari/napari#6025

I think we need to address failing tests here before merge, right?

Mhhh, napari removed the plugin preview service chanzuckerberg/napari-hub#1216. I'll delete the github action.

so, ready to merge?

Yes, looking good to me!